Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make SI damage round naturally and have minimum of 1 #5274

Merged
merged 1 commit into from
Mar 26, 2024

Conversation

Sleet01
Copy link
Collaborator

@Sleet01 Sleet01 commented Mar 22, 2024

Per the ruling here: https://bg.battletech.com/forums/index.php?topic=81913.0
and confirmed on the CGL Discord here: https://discord.com/channels/1157414258133442710/1200547291321339924/1220749492773720195

Aerospace SI damage should be rounded naturally to a minimum of 1.
This patch implements that functionality.

Testing:

  • Ran all 3 projects' unit tests
  • Fired LBX cluster rounds at a variety of Aerospace units with armor lowered or removed to confirm minimum damage value applies

Close #5273

@Sleet01
Copy link
Collaborator Author

Sleet01 commented Mar 23, 2024

Please hold off on this PR until I can confirm that we handle zero-damage heat weapon "damage" correctly, per the follow-up in the linked ruling.

Confirmed.

@Sleet01 Sleet01 added the In Development (Draft) An additional way to mark something as a draft. Make it stand out more. label Mar 23, 2024
@Sleet01
Copy link
Collaborator Author

Sleet01 commented Mar 23, 2024

Confirmed that heat-only weapons do not now apply minimum 1 damage to SI of heat-tracking Aerospace units (to wit, AeroSpaceFighters). Also further confirmed appropriate application of 1 SI damage from single damage clusters such as:

  • LRM-2 launchers that only hit with one missile,
  • Silver Bullet Gauss
  • LRM MML3s only hitting with one missile

This PR is ready to be pulled.

@Sleet01 Sleet01 removed the In Development (Draft) An additional way to mark something as a draft. Make it stand out more. label Mar 23, 2024
@Sleet01 Sleet01 merged commit b44aefb into MegaMek:master Mar 26, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[0.49.19] Make Aero SI take min 1 damage per ruling
2 participants